-
Notifications
You must be signed in to change notification settings - Fork 740
Fix a few examples in docs that are not 2.19 compatible #2581
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@sivel - Should these be backported to other releases so we are showing the correct content in 2.16-2.18 as well? |
|
I'm not sure I would backport in it's entirety, which I suppose also means I'd be fine not backporting. If we did backport, I think only:
The other changes should stay in the older branches. |
Backport to stable-2.19: 💚 backport PR created✅ Backport PR branch: Backported as #2629 🤖 @patchback |
* Fix a few examples in docs that are not 2.19 compatible * remove finalize example of None -> empty string * A few assert cleanups (cherry picked from commit 6a3a90c)
* Fix a few examples in docs that are not 2.19 compatible * remove finalize example of None -> empty string * A few assert cleanups (cherry picked from commit 6a3a90c) Co-authored-by: Matt Martz <[email protected]>
Fix a few examples in docs that are not 2.19 compatible